Skip to content
This repository was archived by the owner on Aug 15, 2025. It is now read-only.

T7678: Remove host-resources from CLI - #57

Closed
natali-rs1985 wants to merge 2 commits into
vyos-legacy:currentfrom
natali-rs1985:T7678
Closed

T7678: Remove host-resources from CLI#57
natali-rs1985 wants to merge 2 commits into
vyos-legacy:currentfrom
natali-rs1985:T7678

Conversation

@natali-rs1985

@natali-rs1985 natali-rs1985 commented Aug 5, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

It will be configured in section "system option host-resources"

Change Summary

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes)
  • Other (please describe): remove CLI section

Related Task(s)

Related PR(s)

vyos/vyos-1x#4647

Proposed changes

How to test

vyos@vyos# /usr/libexec/vyos/tests/smoke/cli/test_vpp.py -k test_19_host_resources
test_19_host_resources (__main__.TestVPP.test_19_host_resources) ... ok

----------------------------------------------------------------------
Ran 1 test in 25.023s

OK

Checklist:

  • I have read the CONTRIBUTING document
  • I have linked this PR to one or more Phabricator Task(s)
  • My commit headlines contain a valid Task id
  • My change requires a change to the documentation
  • I have updated the documentation accordingly

sever-sever
sever-sever approved these changes Aug 5, 2025
View reviewed changes

@sever-sever sever-sever left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The host-resources calculations and CLI are moving to vyos/vyos-1x#4647

@natali-rs1985 natali-rs1985 mentioned this pull request Aug 5, 2025
Closed
10 tasks
@natali-rs1985
T7678: Remove host-resources from CLI
03a0285 
It will be configured in  section "system option host-resources"
@natali-rs1985
natali-rs1985 force-pushed the T7678 branch 2 times, most recently from bf0f29e to 069afd6 Compare August 5, 2025 11:17
@sever-sever

sever-sever commented Aug 8, 2025

Copy link
Copy Markdown
Contributor

This PR should be moved to the vyos-1x after merging vyos/vyos-1x#4650

@natali-rs1985

natali-rs1985 commented Aug 12, 2025

Copy link
Copy Markdown
Contributor Author

Move this changes to vyos-1x repo

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@dmbaturin dmbaturin Awaiting requested review from dmbaturin dmbaturin is a code owner
@jestabro jestabro Awaiting requested review from jestabro
@zdc zdc Awaiting requested review from zdc
1 more reviewer
@sever-sever sever-sever sever-sever approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@natali-rs1985 @sever-sever